Skip to content

Conversation

@SoldierSacha
Copy link

Motivation and Context

How Has This Been Tested?

Breaking Changes

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update

Checklist

  • I have read the MCP Documentation
  • My code follows the repository's style guidelines
  • New and existing tests pass locally
  • I have added appropriate error handling
  • I have added or updated documentation as needed

Additional context

…for-oauth-client-credentials

Add OAuth client credentials grant
…mplement-client-credentials-support

Fix auth registration for client credentials
…mentation-of-client-credentials-flow

Refactor auth helper methods
Fix failing tests and pyright
…eerror-in-pytest-test-case

Fix OAuth tests using async fixtures
Fix resumption token update logic
Add OAuth token exchange support
Add token exchange docs
…change-implementation-for-test-failures

Fix token exchange metadata tests
Reported-by: sachabaniassad
Fix pyright case issues in tests and example
SoldierSacha and others added 28 commits September 23, 2025 16:49
…-test-for-client-registration

Allow additional grant types during client registration
…ge-conflicts-in-sdk-fork

Resolve OAuth auth flow merge conflicts
…ionreseterror-in-stdio_client-test

Handle closed stdin in stdio client
# Conflicts:
#	src/mcp/client/auth/oauth2.py
#	src/mcp/shared/auth.py
#	tests/client/test_auth.py
#	tests/issues/test_88_random_error.py
#	tests/shared/test_streamable_http.py
…hub-merge-conflicts-for-oauth-support

Resolve upstream merge conflicts while preserving OAuth extensions
Add explicit None checks for client_id fields before passing to functions
that expect str. This fixes type errors where str | None was being passed
to parameters that require str.

Changes:
- simple_auth_provider.py: Add client_id validation in exchange_client_credentials and exchange_token
- oauth2.py: Add client_id check at start of _apply_client_auth method
- test_auth_integration.py: Add assertions for client_id not being None in test mock methods

This ensures proper type safety and prevents potential None dereference errors.
@SoldierSacha SoldierSacha changed the title Claude/fix oauth2 type errors 011 c un6g biipfcg g jx akt ra v (Closed) Nov 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants